Conversation
browniebroke
force-pushed
the
refactor/pyproject.toml
branch
4 times, most recently
from
d993427 to
7cc49d5
Compare
browniebroke
force-pushed
the
refactor/pyproject.toml
branch
from
7cc49d5 to
be922d7
Compare
There was a problem hiding this comment.
Pull request overview
This PR migrates the project's dependency management from requirements.txt + pip to uv with pyproject.toml and uv.lock. The CI workflow is updated accordingly.
Changes:
- Replaced
requirements.txtwithpyproject.tomlanduv.lockfor dependency management - Updated
.github/workflows/ci.ymlto useastral-sh/setup-uvanduv sync/uv runinstead ofsetup-pythonandpip
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
pyproject.toml |
New project metadata and dependency definitions replacing requirements.txt |
uv.lock |
Auto-generated lock file pinning all transitive dependencies |
requirements.txt |
Removed in favor of pyproject.toml |
.github/workflows/ci.yml |
Updated CI to use uv tooling instead of pip/setup-python |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| readme = "README.md" | ||
| requires-python = "==3.13.*" | ||
| dependencies = [ | ||
| "coreapi==2.3.3", |
There was a problem hiding this comment.
The coreapi dependency was not present in the original requirements.txt and is not imported anywhere in the codebase. This appears to be an unintentional addition. It should be removed from the dependencies list to keep the migration a 1:1 replacement.
Just a suggestion - replace
requirements.txtbypyproject.tomlanduv.lockIf we want to go ahead, the docs will need updating too